Skip to content

RBAC/Admin Console Ulmo docs#1351

Merged
sarina merged 3 commits intomainfrom
sarina/ulmo-authz-console
Jan 7, 2026
Merged

RBAC/Admin Console Ulmo docs#1351
sarina merged 3 commits intomainfrom
sarina/ulmo-authz-console

Conversation

@sarina
Copy link
Contributor

@sarina sarina commented Dec 11, 2025 •
edited
Loading

sarina
sarina commented Dec 14, 2025
View reviewed changes
@sarina sarina force-pushed the sarina/ulmo-authz-console branch 3 times, most recently from 3723365 to d21e7d6 Compare December 15, 2025 17:11
@sarina sarina marked this pull request as ready for review December 15, 2025 17:12
mariajgrimaldi
mariajgrimaldi reviewed Dec 15, 2025
View reviewed changes
MaferMazu
MaferMazu reviewed Dec 15, 2025
View reviewed changes
Copy link
Contributor

@MaferMazu MaferMazu left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for this PR @sarina and @gviedma-aulasneo ✨

I still need to review the ulmo_rp and the how-to of adding a user, I'll do tomorrow, but I am sharing what I have regarding the rest.

Note: probably in the future we want to unify the roles and permissions page here or in authz, but it would be great to have only one page so we don't repeat ourselves and make it easier to maintain when we add more default roles and permissions.

@sarina sarina mentioned this pull request Dec 16, 2025
Merged
MaferMazu
MaferMazu reviewed Dec 17, 2025
View reviewed changes
Copy link
Contributor

@MaferMazu MaferMazu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I just finished reviewing the ulmo_rp and the how-to to add roles.

In general, it's looking well. I only added some suggestions.

Thanks again for this work.

@sarina sarina force-pushed the sarina/ulmo-authz-console branch from d21e7d6 to 39f7e80 Compare December 17, 2025 18:11
@MaferMazu MaferMazu linked an issue Dec 18, 2025 that may be closed by this pull request
Update content libraries roles and permissions documentation openedx/openedx-authz#161
Closed
@sarina sarina mentioned this pull request Dec 19, 2025
Merged
@sarina sarina force-pushed the sarina/ulmo-authz-console branch from 39f7e80 to 6c37706 Compare December 19, 2025 21:20
@sarina
Copy link
Contributor Author

sarina commented Dec 19, 2025

@MaferMazu @mariajgrimaldi I removed the Library R&P pages and did cross-references to the authz docs in a few places. I did wholesale copy-paste the table from https://docs.openedx.org/projects/openedx-authz/en/latest/concepts/core_roles_and_permissions/content_library_roles.html#roles-and-permissions-summary-table into the Ulmo release note because I couldn't embed it unfortunately, but since it's point-in-time documentation I think it's okay. Can I go ahead and merge this?

MaferMazu
MaferMazu approved these changes Dec 23, 2025
View reviewed changes
Copy link
Contributor

@MaferMazu MaferMazu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Everything looks good to me. (only a minor comment no blocker)

Thanks @sarina ✨

source/educators/how-tos/course_development/add_library_content_to_a_course.rst Outdated
before it can be used in a Course.

A course author must have at least Read-only permissions to reuse the content.
A course author must have at least the **Library User** role to reuse content.
Copy link
Contributor

@MaferMazu MaferMazu Dec 23, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
A course author must have at least the **Library User** role to reuse content.
A course author must have at least the **Library User** role to reuse content, *or*,
the library should be published with the **Allow Public Read** permission.

Copy link
Contributor

@MaferMazu MaferMazu Dec 23, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(no blocker)

@sarina sarina force-pushed the sarina/ulmo-authz-console branch from 6c37706 to 6ca6a81 Compare January 7, 2026 14:26
@sarina sarina enabled auto-merge (rebase) January 7, 2026 14:26
@sarina sarina merged commit 2a0b6dc into main Jan 7, 2026
2 checks passed
@sarina sarina deleted the sarina/ulmo-authz-console branch January 7, 2026 14:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

4 more reviewers

@rodmgwgu rodmgwgu rodmgwgu left review comments

@MaferMazu MaferMazu MaferMazu approved these changes

@mariajgrimaldi mariajgrimaldi mariajgrimaldi approved these changes

@gviedma-aulasneo gviedma-aulasneo gviedma-aulasneo approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Update content libraries roles and permissions documentation

5 participants

@sarina @MaferMazu @mariajgrimaldi @rodmgwgu @gviedma-aulasneo